Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Apply IP allow list to CMS application regardless of environment type #1270

Merged
merged 8 commits into from
Mar 3, 2025

Conversation

A-Ashiq
Copy link
Contributor

@A-Ashiq A-Ashiq commented Feb 25, 2025

This PR does the following:

  • Enforces the IP allow list on the CMS WAF regardless of whether its dev or not

@A-Ashiq A-Ashiq requested a review from a team as a code owner February 25, 2025 14:35
Copy link

github-actions bot commented Feb 25, 2025

unit test coverage report

Title Lines Statements Branches Functions
lambda-producer-handler Coverage: 100%
100% (41/41) 25% (2/8) 100% (8/8)
lambda-db-password-rotation Coverage: 100%
100% (26/26) 40% (4/10) 100% (4/4)
lambda-alarm-notification Coverage: 100%
100% (23/23) 100% (5/5)
legacy-dashboard-redirect-viewer-request Coverage: 100%
100% (16/16) 100% (4/4) 100% (3/3)
public-api-cloud-front-viewer-request Coverage: 100%
100% (18/18) 100% (12/12) 100% (3/3)
Title Tests Skipped Failures Errors Time
lambda-producer-handler 16 0 💤 0 ❌ 0 🔥 0.814s ⏱️
lambda-db-password-rotation 5 0 💤 0 ❌ 0 🔥 0.629s ⏱️
lambda-alarm-notification 5 0 💤 0 ❌ 0 🔥 0.675s ⏱️
legacy-dashboard-redirect-viewer-request 10 0 💤 0 ❌ 0 🔥 0.546s ⏱️
public-api-cloud-front-viewer-request 11 0 💤 0 ❌ 0 🔥 0.497s ⏱️

@A-Ashiq A-Ashiq merged commit 6d52cba into main Mar 3, 2025
9 checks passed
@A-Ashiq A-Ashiq deleted the task/add-ip-allow-list-to-cms/CDD-2513 branch March 3, 2025 10:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants